Add all Champions learnsets#1532
Conversation
| @@ -1,4 +1,5 @@ | |||
| id,identifier | |||
| 0,null | |||
There was a problem hiding this comment.
I would say this is more of a move tutor situation. Plus, I don't think the API can handle IDs below 1. Making a local version of the database shows it assigns them to zygarde-cube instead.
Do you mean that Mega Meowstic has a different learnset depending on if it's male or female? If so, we should make a pokemon resource for each of them and add the relevant move data to each. Otherwise, we should add the move data to the single Mega Meowstic. |
2dea40e to
58a7af4
Compare
Yes. Like all Mega-Evolved Pokémon, Mega Meowstic has the exact same learnset as its base form. But male and female Meowstic have different learnsets, so male and female Mega Meowstic also have different learnsets. Splitting Mega MeowsticI've updated In the documentation for Pokémon forms,
The documentation does actually note "Multiple forms may have equal order, in which case they should fall back on sorting by name", so theoretically it may actually not be necessary to insert a new distinct |
Change description
Add all Pokémon Champions learnsets. Data from https://github.com/projectpokemon/champout/blob/main/masterdata/waza_learn.json
Champions is just a battle sim where you can freely swap moves around, so it doesn't really have a concept of "methods" for learning moves. My solution was to define the new
pokemon_move_method"null" at index 0. If it would be preferable to make this a new positive index number, or something else entirely, let me know and I can swap it over.Notes:
pokemon_idvalues for these forms, so I've had to condense these into single entries per Pokémon, but that doesn't make any functional difference.pokemon_idfor Mega Meowstic. Since there's no good way to mark this, I've simply not included any moves for Mega Meowstic at all.AI coding assistance disclosure
No AI was used in the creation of this PR.
Contributor check list